✅ Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.42%. Comparing base (0e60143) to head (1e66bfb).

@@            Coverage Diff             @@
##             main    #1096      +/-   ##
==========================================
+ Coverage   77.40%   77.42%   +0.02%     
==========================================
  Files          56       56              
  Lines        2288     2286       -2     
  Branches      258      258              
==========================================
- Hits         1771     1770       -1     
+ Misses        487      486       -1     
  Partials       30       30              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jescalada - thanks for the PR ❤️

Perhaps it makes sense to display the error message outside of the table in a pop-up? Open until closed by the user?

I modified this so that it uses one of the predefined error components (and got the message out of the tabs box):

There's still an issue with a non-descriptive error when JWT is enabled but the config is missing.

I've tried to find better-looking error displays (such as custom-styled Snackbars, and other components), but it seems MUI v4 doesn't support some of the best ones.

In later versions of MUI we can use these Alert components or combine them with Snackbars to make it look better.

There's this issue for upgrading MUI to v5 #1023 which I'll take a look at soon. It might be worth modernizing the frontend a bit once the dependency upgrades are done. (Note: Upgrading directly to v7 is not possible as each new major version introduces various breaking changes - plus the automatic migration tools only go up one major version at a time)